Thanks for this PR @D3X!

Do you happen to have a screenshot of what a report looks like (before and after) when using this?

What I'm looking for is that the truncation doesn't happen at the end of the name, but rather in the middle or start.

To illustrate.

I don't want these:

tests/component/datepicker_test.py::test_picker_using_mouse
tests/component/datepicker_test.py::test_picker_using_keyboard

to become:

tests/component/datepicker_test.py::test_picker_using
tests/component/datepicker_test.py::test_picker_using

but rather:

tests/.../datepicker_test.py::test_picker_using_mouse
tests/.../datepicker_test.py::test_picker_using_keyboard

or:

.../component/datepicker_test.py::test_picker_using_mouse
.../component/datepicker_test.py::test_picker_using_keyboard

Hope that makes sense! 😊

Oh wait... You're talking about filename and not testname... right? 🙈

@BeyondEvil um, yes, filenames. Perhaps I should've worded that differently :)
This truncates test names when used for asset filenames. Only to avoid ending up with filenames that are longer than what the FS supports.

(force-pushed a fix for flake8 complaints)

@D3X Nothing wrong with your wording, I just have to learn how to read. 😂

This is awesome! Just because I'm curious, did you see an issue where the original functionality broke?

As soon as tests pass, I'll approve! 👍

@BeyondEvil yes, our test suite was broken because of that, that's why I created this PR.

Nested test classes + long test method names + parametrization = looooong test names = OSError :)

OSError: [Errno 36] File name too long: '/home/circleci/project/selenium_html/assets/core/tests/selenium_tests/lawfirm_tests/law_firm_manual_matters/test_law_firm_manual_matters_on_business.py::TestLawFirmManualMattersOnBusiness::test_closed_law_firm_manual_matter_can_enter_accruals_for_this_month[INVOICE_VIEWS:False-NEW_LAW_FIRM_MATTERLIST:True-LAW_FIRM_ACCESS_CONTROL:True]01_0a7bf5cac9650399d9bdd7618e2d7e26.txt'

@D3X WOW! That's... impressive! 😊

@D3X When this is merged I'll make a hotfix release tonight or tomorrow, because this is a bug. 👍

I'll leave the honors to you @D3X.

Unless you take too long. ;)

@BeyondEvil I don't think I have access to merge, so go ahead :)

@D3X Version 1.21.1 is out with your fix! Thanks! 👍